Skip to content

[presets] Escalate C++ "unused" warnings in macOS smoke test #81284

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 14, 2025

Conversation

AnthonyLatsis
Copy link
Collaborator

This is a modest attempt to make our builds more warning-free. Over time, we can add more warning groups and extrapolate this to Swift sources.

@AnthonyLatsis
Copy link
Collaborator Author

@swift-ci please smoke test macOS

@AnthonyLatsis AnthonyLatsis requested a review from edymtt May 7, 2025 14:01
Copy link
Contributor

@edymtt edymtt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me with regard to CMake and preset changes.

@AnthonyLatsis AnthonyLatsis force-pushed the no-warnings-asserts branch from 5f7b588 to 9a1995a Compare May 7, 2025 16:24
@AnthonyLatsis
Copy link
Collaborator Author

@swift-ci please smoke test macOS

@AnthonyLatsis AnthonyLatsis force-pushed the no-warnings-asserts branch from 9a1995a to e7237fd Compare May 13, 2025 15:15
@AnthonyLatsis
Copy link
Collaborator Author

@swift-ci please smoke test macOS

@AnthonyLatsis
Copy link
Collaborator Author

@swift-ci please smoke test

@AnthonyLatsis AnthonyLatsis enabled auto-merge May 14, 2025 17:53
@AnthonyLatsis AnthonyLatsis disabled auto-merge May 14, 2025 19:34
@AnthonyLatsis AnthonyLatsis enabled auto-merge May 14, 2025 19:34
@AnthonyLatsis AnthonyLatsis merged commit 0777dbd into swiftlang:main May 14, 2025
3 checks passed
@AnthonyLatsis AnthonyLatsis deleted the no-warnings-asserts branch May 14, 2025 22:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants